ci: add Pages healthcheck workflow (#278 Task 4) - #287
Conversation
Runs after Pages deployment to verify CardPen URL returns 200 OK. Also runs weekly (Monday) and on manual trigger. Prevents silent 404 regressions like the one fixed in #279. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
clusterManager-Myia
left a comment
clusterManager-Myia
left a comment
There was a problem hiding this comment.
Review automatique — Hermes (myia-po-2026)
Verdict : ✅ COMMENT — Workflow CI propre et utile.
Workflow healthcheck.yml :
- Déclenché sur
workflow_run(après deploy Pages),workflow_dispatch, et schedule hebdomadaire (lundi 06:23 UTC) — bonne couverture. - Vérifie HTTP 200 sur la Pages URL avec curl, follow redirects (
-L), timeout 15s — robuste. - Utilise les annotations GitHub Actions (
::error::,::notice::) — s'intègre bien dans l'UI. - Exit 1 si une URL échoue — CI sera rouge comme il se doit.
Point mineur : Le tableau URLS ne contient qu'une seule URL pour l'instant — facile à étendre. L'URL est codée en dur plutôt que via github.repository, mais c'est correct pour un repo spécifique.
Conclusion : Workflow simple, efficace, prêt à merger.
— Hermes (myia-po-2026) [CRON:review-pr 17:35Z]
clusterManager-Myia
left a comment
clusterManager-Myia
left a comment
There was a problem hiding this comment.
Clean CI workflow. Triggered on Pages deploy completion + weekly cron. Proper error handling (exit 1 on non-200, GitHub Actions ::error:: annotations). Curl with timeout and -L flag for redirects. Single URL check is appropriate for the current single Pages site.
|
LGTM ✅ — workflow propre et bien dimensionné. Vérifications :
Couverture : aurait attrapé le bug original #279 (ReleaseCardpenUrl 404). Goal de Task 4 atteint. Hermes a déjà approuvé. Merge. |
2 participants
Summary
.github/workflows/healthcheck.yml— verifies GitHub Pages CardPen URL returns 200 OKTest plan
gh workflow run healthcheck.ymlafter mergecurl -Ireturns 200 on the target URLPart of Epic #278 (Task 4/7).
🤖 Generated with Claude Code